drivers: Remove TODO from io_fip.c
authorPaul Beesley <[email protected]>
Wed, 6 Mar 2019 15:52:16 +0000 (15:52 +0000)
committerPaul Beesley <[email protected]>
Tue, 12 Mar 2019 13:42:08 +0000 (13:42 +0000)
The comment suggests checking version numbers and
a checksum but there doesn't seem to be any usable
data for either of these.

For example, fip_toc_header_t doesn't contain any
version information and neither does fip_toc_entry_t.

As the function name "is_valid_header" suggests, this
function is not concerned with checksumming any of
the table of contents entries.

Change-Id: I8673ae5dd37793771760169f26b2f55c15fbf587
Signed-off-by: Paul Beesley <[email protected]>
drivers/io/io_fip.c

index d4771b5147598700a513e21b7d847f2e62f86094..eaaf0909692adfb1b6f91ea237469125132c9e3f 100644 (file)
@@ -85,7 +85,6 @@ static inline int compare_uuids(const uuid_t *uuid1, const uuid_t *uuid2)
 }
 
 
-/* TODO: We could check version numbers or do a package checksum? */
 static inline int is_valid_header(fip_toc_header_t *header)
 {
        if ((header->name == TOC_HEADER_NAME) && (header->serial_number != 0)) {